Skip to content

fix(models): align RenderWidget.magnet with backend MAGNET enum value#197

Merged
PHKenny merged 2 commits into
mainfrom
development
May 16, 2026
Merged

fix(models): align RenderWidget.magnet with backend MAGNET enum value#197
PHKenny merged 2 commits into
mainfrom
development

Conversation

@PHKenny
Copy link
Copy Markdown
Contributor

@PHKenny PHKenny commented May 16, 2026

📋 Summary

  • RenderWidget had a stale MAGNET_SENSOR JSON value that never matched the backend — magnet widgets would silently fall back to unknown at runtime.
  • Corrected the variant name (magnetSensormagnet) and its @JsonValue ('MAGNET_SENSOR''MAGNET') to match the backend Widget enum exactly.
  • Bumped version to 3.8.16 and updated CHANGELOG.md.

🐛 Fixes

  • RenderWidget.magnet: changed @JsonValue from 'MAGNET_SENSOR' to 'MAGNET' to match the backend Widget enum and prevent silent fallback to unknown.

🔧 Chore / Config

  • Bumped pubspec.yaml version from 3.8.15 to 3.8.16.
  • Updated CHANGELOG.md with the 3.8.16 entry.

🤖 Generated with Claude Code

PHKenny and others added 2 commits May 16, 2026 15:31
Renamed `magnetSensor` to `magnet` and corrected the @jsonvalue from
'MAGNET_SENSOR' to 'MAGNET' to match the backend Widget enum.
Regenerated models.g.dart.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

📋 Changelog Summary

This PR includes the following changes:

🐛 Bug Fixes

  • align RenderWidget.magnet with backend MAGNET enum value (2193447 by @PHKenny, Claude Sonnet 4.6)

🤖 Auto-generated by changelog workflow

@github-actions
Copy link
Copy Markdown

📊 Test Coverage Report

🎯 Dart/Flutter

Metric Value
Coverage 11.4%
Lines covered 14939 / 450
Lines missing -14489

🤖 Generated by coverage workflow

@PHKenny PHKenny merged commit 9d3f302 into main May 16, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant